Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[PoC]: replaces yarn with pnpm #148

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

[PoC]: replaces yarn with pnpm #148

wants to merge 2 commits into from

Conversation

andresgnlez
Copy link
Member

run corepack enable && corepack prepare && pnpm install and profit!

@andresgnlez andresgnlez requested a review from davidsingal June 15, 2023 11:31
@vercel
Copy link

vercel bot commented Jun 15, 2023 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
front-end-scaffold-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 15, 2023 1:22pm

@davidsingal
Copy link
Contributor

Before merging, I think we should tell the benefits of pnpm vs yarn. Or at least, a small talk about why you want to suggest moving. Then, I would like junior profiles to test it because it could be a obstacle in the daily work.

Base automatically changed from v1 to main August 4, 2023 10:04
@andresgnlez andresgnlez changed the title replaces yarn with pnpm [PoC]: replaces yarn with pnpm Aug 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@davidsingal davidsingal Awaiting requested review from davidsingal

At least 1 approving review is required to merge this pull request.

Assignees

@davidsingal davidsingal

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@andresgnlez @davidsingal